Turn back on new GPX writer. With this change, it passes for me on Centos 6.3

running in a VM.

The new GPX writer isn't actually *done* yet, but it'll get done only by
suffering...Pain builds character.

git-svn-id: http://gpsbabel.googlecode.com/svn/trunk@4318 f51c46e8-681c-474f-0cfe-069cfd0219fb

diff --git a/gpsbabel/Makefile.in b/gpsbabel/Makefile.in
index 701ef69d99bd2c55f24aba1459f382b6f755ffdb..2a30c6b664058114e1ded28e46d51a31428b408b 100644 (file)
--- a/gpsbabel/Makefile.in
+++ b/gpsbabel/Makefile.in
@@ -52,7 +52,7 @@ LRELEASE=@LRELEASE@
 #DEBUGGING=-g $(EXTRA_DEBUGGING)
 # add -DDEBUG_MEM to turn on memory allocation logging
 GBCFLAGS=$(EXTRA_CFLAGS) $(DEBUGGING) -I$(srcdir) -I$(QT_INC) \
-       $(OPTIMIZATION) -DHAVE_CONFIG_H @CFLAGS@ -DNEWTIME -DOLDGPX
+       $(OPTIMIZATION) -DHAVE_CONFIG_H @CFLAGS@ -DNEWTIME -UOLDGPX
 LDFLAGS=$(EXTRA_LDFLAGS) @LDFLAGS@
 PREFIX=@prefix@
 INSTALL_DIR=$(DESTDIR)/$(PREFIX)

diff --git a/gpsbabel/gpx.cc b/gpsbabel/gpx.cc
index 2188e843c584f75ba82f183122a0cdd9d412f6be..6b905f1ce09c3aab7a18627beb3142eb559cde14 100644 (file)
--- a/gpsbabel/gpx.cc
+++ b/gpsbabel/gpx.cc
@@ -2235,6 +2235,12 @@ gpx_write(void)
   gbfprintf(ofd, "  creator=\"" CREATOR_NAME_URL "\"\n");
   gbfprintf(ofd, "  xmlns:xsi=\"http://www.w3.org/2001/XMLSchema-instance\"\n");
 #else
+  // FIXME: This write of a blank line is needed for Qt 4.6 (as on Centos 6.3)
+  // to include just enough whitespace between <xml/> and <gpx...> to pass
+  // diff -w.  It's here for now to shim compatibility with our zillion 
+  // reference files, but this blank link can go away some day.
+  writer.writeCharacters("\n");
+
   writer.setAutoFormatting(true);
   writer.writeStartElement("gpx");
   writer.writeAttribute("\n  version", gpx_wversion);